Skip to content

[NVVM] Expose nvvm version detection in cuda.bindings.utils.#1837

Open
abhilash1910 wants to merge 11 commits intoNVIDIA:mainfrom
abhilash1910:nvvm_fix
Open

[NVVM] Expose nvvm version detection in cuda.bindings.utils.#1837
abhilash1910 wants to merge 11 commits intoNVIDIA:mainfrom
abhilash1910:nvvm_fix

Conversation

@abhilash1910
Copy link
Copy Markdown
Contributor

Description

Fixes issue #1457 . Provides an api call to cuda.bindings.utils to check the version of nvvm.
@leofang @rwgk pinging for review.

@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot bot commented Mar 31, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@abhilash1910
Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@abhilash1910 abhilash1910 changed the title [NVVM]Expose nvvm version detection in cuda.bindings.utils. [NVVM][Fix] Expose nvvm version detection in cuda.bindings.utils. Mar 31, 2026
@abhilash1910 abhilash1910 changed the title [NVVM][Fix] Expose nvvm version detection in cuda.bindings.utils. [NVVM] Expose nvvm version detection in cuda.bindings.utils. Mar 31, 2026
@rwgk rwgk added the cuda.bindings Everything related to the cuda.bindings module label Mar 31, 2026
@rwgk rwgk added this to the cuda.bindings backlog milestone Mar 31, 2026
@rwgk rwgk added enhancement Any code-related improvements P1 Medium priority - Should do labels Mar 31, 2026
@@ -0,0 +1,93 @@
# SPDX-FileCopyrightText: Copyright (c) 2026-2027 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove -2027

@rparolin rparolin removed this from the cuda.bindings backlog milestone Apr 2, 2026
@abhilash1910
Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@abhilash1910
Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@rwgk rwgk added this to the cuda.bindings next milestone Apr 7, 2026
@rwgk rwgk added awaiting-response Further information is requested and removed awaiting-response Further information is requested labels Apr 7, 2026
@rwgk
Copy link
Copy Markdown
Collaborator

rwgk commented Apr 7, 2026

/ok to test c13c860

@rwgk
Copy link
Copy Markdown
Collaborator

rwgk commented Apr 7, 2026

/ok to test e2afa85

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

@rwgk
Copy link
Copy Markdown
Collaborator

rwgk commented Apr 7, 2026

This doesn't work with the cuda-bindings from the 12.9.x backport branch, e.g.:

https://github.com/NVIDIA/cuda-python/actions/runs/24091305075/job/70279241852?pr=1837

tests/test_program.py:76: in _check_nvvm_arch
    from cuda.bindings.utils import check_nvvm_compiler_options
E   ImportError: cannot import name 'check_nvvm_compiler_options' from 'cuda.bindings.utils' (/opt/hostedtoolcache/Python/3.10.20/x64/lib/python3.10/site-packages/cuda/bindings/utils/__init__.py)

@leofang wrote under #1457:

We should expose it to cuda.bindings.utils, similar to the PTX version helpers.

That makes using it from cuda_core/tests/ a little tricky: it'll only be in future cuda-bindings releases.

@leofang how would you handle this situation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindings Everything related to the cuda.bindings module enhancement Any code-related improvements P1 Medium priority - Should do

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants